Skip to content

Improve test coverage#521

Open
s-hadinger wants to merge 1 commit intoberry-lang:masterfrom
s-hadinger:improve_coverage_tests
Open

Improve test coverage#521
s-hadinger wants to merge 1 commit intoberry-lang:masterfrom
s-hadinger:improve_coverage_tests

Conversation

@s-hadinger
Copy link
Contributor

Improve coverage tests (aggregated PR):

  • Fix warnings in lcov when running make test; silence normal warnings
  • Enhance test of math module for 100% code coverage
  • Enhance test of module module from 50% to 80% code coverage
  • Enhance test of range class for 100% code coverage
  • Improve code coverage for list and map
  • Achieve 100% code coverage for module introspect and mark be_raise() as LCOV_EXCL_LINE to not account in gcov
  • Improve code coverage for be_vm.c
  • Ensure 100% coverage for time module
  • Ensure 100% coverage for gc module
  • Simplify gc.allocated() that does not need to use real if int is already 64 bits
  • More generally improve comments in other test files

No functional change.

Output for make test:

[Run Testcases...]
Filter suppressions:
  region:
    2 instances
Message summary:
  3 ignore messages:
    gcov: 2
    unsupported: 1
run testcase: gc.be
run testcase: global.be
run testcase: bitwise.be
run testcase: compiler.be
run testcase: list.be
run testcase: virtual_methods.be
run testcase: division_by_zero.be
run testcase: json_advanced.be
run testcase: time.be
run testcase: class_static.be
run testcase: introspect.be
run testcase: checkspace.be
run testcase: os.be
run testcase: subobject.be
run testcase: class_const.be
run testcase: range.be
run testcase: walrus.be
run testcase: be_api.be
run testcase: vm_coverage.be
run testcase: cond_expr.be
run testcase: member_indirect.be
run testcase: bool.be
run testcase: lexer.be
run testcase: assignment.be
run testcase: reference.be
run testcase: exceptions.be
run testcase: bytes_b64.be
run testcase: debug.be
value type <class>, attributes:
run testcase: class.be
run testcase: compound.be
run testcase: bytes_fixed.be
run testcase: lexergc.be
run testcase: introspect_ismethod.be
run testcase: map.be
run testcase: super_auto.be
run testcase: parser.be
run testcase: function.be
run testcase: closure.be
run testcase: vararg.be
run testcase: comptr.be
run testcase: json_test_stack_size.be
run testcase: json.be
run testcase: int.be
run testcase: super_leveled.be
run testcase: bytes.be
run testcase: relop.be
run testcase: call.be
run testcase: suffix.be
run testcase: virtual_methods2.be
run testcase: for.be
run testcase: math.be
run testcase: module.be
run testcase: string.be
run testcase: overload.be
123
test results: 54 total, 0 failed (all tests passed).
Filter suppressions:
  region:
    2 instances
Message summary:
  3 ignore messages:
    gcov: 2
    unsupported: 1
Message summary:
  no messages were reported
Excluding /Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/math.h
Removed 1 files
Writing data to final.info
Summary coverage rate:
  source files: 41
  lines.......: 77.7% (9579 of 12336 lines)
  functions...: 81.8% (837 of 1023 functions)
Message summary:
  no messages were reported
Overall coverage rate:
  source files: 41
  lines.......: 77.7% (9579 of 12336 lines)
  functions...: 81.8% (837 of 1023 functions)
Message summary:
  no messages were reported

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant